Do you have some failing tests? (Travis only seems to be complaining about standard) How can I repro?

@noffle thanks for taking a look!

I've added the http tests with daemon on that should throw an error. I have just been manually trying to add a file with the go/js-ipfs daemons running.

Hey @nginnever, I noticed a couple of things:

1. It looks like js-ipfs is returning a Buffer for HTTP object/data and go-ipfs returns a stream. We really ought to update js-ipfs and interface-ipfs-core to use streams consistently.
2. For your HTTP tests: ipfs.files.add returns { path: '...', node: DAGNode }, not { Name: '...', Hash: '...' }

1. It looks like js-ipfs is returning a Buffer for HTTP object/data and go-ipfs returns a stream. We really ought to update js-ipfs and interface-ipfs-core to use streams consistently.

My thoughts exactly on the inconsistency in object. I will take a look at those commands and see about returning streams instead of the buffer (which i noticed that buffer isn't even the same data buffer that gets returned from the dag service so not sure what's going on there.)

2

Ahh good catch, still some old tests slipping in before the changes to return type.

@noffle I've created a branch on js-ipfs-api that updates get-dagNode.js to check the response from the daemons for buffer or stream.

Though the issue is still as mentioned above that sometimes the response from object/data is outputting a slightly different buffer causing the dag node to multihash to the incorrect value.

If you run the tests now you will see this issue throw an error.

Seems like the core gets the proper data from the dagservice here.

but not when returned here

After some investigating I found that this line in the object resource is the culprit. Removing the toString seems to fix.

This PR is now waiting on

• js-ipfs-api upgrade
• js-ipfs remove toString

@nginnever how is PR coming along?

Once #420 is merged, could you rebase this PR @nginnever ?

beep boop, #420 is merged :)

beep boop, needing this one for #432 :)

Needs ipfs-inactive/interface-js-ipfs-core#71

@victorbjelkholm got #323 (comment) released

Upgrade and merge conflict resolution happening here: #469

Ok, so @victorbjelkholm finished up the integration and got all tests passing for get, cat and add 👏👏👏👏. Thank you @nginnever and @noffle for having kicked off all of this work 🌟 👌🏽

Now it is all about the big rebase (because js-ipfs codebase changed with the migration to pull-streams). This is happening here: #469 by @dignifiedquire

Closing this PR, let's track the rest on the rebase PR (which is already pointing to master) #469